Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

Fix Typos #1662

Closed
wants to merge 2 commits into from
Closed

Fix Typos #1662

wants to merge 2 commits into from

Conversation

Dimitrolito
Copy link

This pull request addresses minor typos in the documentation for better readability and consistency.

Changes Made:

  1. README.md:

    • Corrected the article from "(zk)-Ethereum" to "a (zk)-Ethereum" for grammatical accuracy.
  2. cairo_precompiles.md:

    • Replaced "a uint256" with "an uint256" to reflect proper article usage based on pronunciation.

Time Spent

Approximately 0.25 days.

Pull Request Type

  • Documentation content changes

Current Behavior

The documentation contained minor grammatical errors that could potentially cause confusion or reduce the professionalism of the text.

New Behavior

The updated text improves grammatical accuracy and clarity without changing the technical content.

Breaking Changes

  • Yes
  • No

@Eikix
Copy link
Member

Eikix commented Jan 8, 2025

Sorry for the late reply and lack of review. We've been pausing the development of kakarot for now.

I'm cognizant that you spent some time doing this PR, so thank you. I will have to close it since I am cleaning a bit the repo and preparing for archival.

Archiving the repo will solve issue of people working on stuff while we are not accepting new PRs.
Thanks for understanding

@Eikix Eikix closed this Jan 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants